Skip to content

[SecurityBundle] Document switch_user.stateless firewall option #9390

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 5, 2018

Conversation

chalasr
Copy link
Member

@chalasr chalasr commented Mar 4, 2018

Fixes #8447.
Note for mergers: this should be removed in 4.0, the switch_user.stateless is set according to the firewall stateless option value there so it just works naturally and the config option is useless.

@@ -66,6 +66,11 @@ firewall listener:
),
));

.. tip::

For using the ``switch_user` listener in a ``stateless`` firewall, set the
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor thing: in switch_user there's a missing closing backtick,

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed thanks!

@javiereguiluz
Copy link
Member

Thanks for contributing this and thanks for leeting us know how to merge it. Cheers!

@javiereguiluz javiereguiluz merged commit 2d71493 into symfony:3.4 Mar 5, 2018
javiereguiluz added a commit that referenced this pull request Mar 5, 2018
…option (chalasr)

This PR was merged into the 3.4 branch.

Discussion
----------

[SecurityBundle] Document switch_user.stateless firewall option

Fixes #8447.
Note for mergers: this should be removed in 4.0, the `switch_user.stateless` is set according to the firewall `stateless` option value there so it just works naturally and the config option is useless.

Commits
-------

2d71493 Document switch_user.stateless firewall option
@chalasr chalasr deleted the switchuser-stateless branch March 6, 2018 01:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants